Add a test for ::row-changed vs node refs

Test that filter models propagate ::row-changed if there is
an external reference on the node, and not otherwise. This
is showing up in buggy icon view behaviour, where the icon
view is not redrawing if the content changes in a model that
is below a filter model.

gtkwindow: Disconnect GdkSeat::device-removed callback on finalize

Otherwise dangling callbacks may lead to crashes.

gl renderer: Fix shadow node child offset

We shouldn't apply the shadow offset when drawing the child offscreen,
instead apply it afterwards when we draw it to the current render
target.

testsuite: Add another gl renderer test case

linkbutton: Only set widget cursor once

inspector: Allow picking insensitive widgets

The default gtk_widget_contains/gtk_widget_pick don't consider
insensitive widgets.

csswidgetnode: Avoid some unnecessary work

The later code would just ref both styles and not do anything else.

widget-factory: Fix some layout issues

We have to explicitly set some of these to their default values so
expand-set is TRUE and the boxes stop propagating their expansion state
up the hierarchy.

textview: Remove some stray debugging output

Leftover from 7046463b880c79b227594a2b35a6c625119d286e

Simplify all ui files again

Now even simpler!

builder-tool: Try harder to find types

If g_type_from_name fails, try gtk_builder_get_type_from_name
before giving up. This fixes cases like GtkPrinterOptionWidget.

window: Ignore gtk_window_close calls from close-request handlers

widget: Always assign values to out parameter in translate_coordinates

safe guard against people using the returned coordinate values without
checking the return value of gtk_widget_translate_coordinates.

widget: Initialize a local variable

We will later pass this variable on to some other functions, so be safe
against them using the value.

aboutdialog: Only close on delete events

Clicking on the Credits button should not close the dialog.

fontchooswerwidget: Center spinbutton next to scales vertically

gl renderer: Render GL textures upside down

These come from an FBO so we need to flip the texture coords on the y
axis, just like we do everywhere else.

Revert "inspector: Stop adding event controllers to the object tree"

This reverts commit 8f3cb2658a79d20912b6f2ec96bc7e6bbdaeb067.

This didn't quite work out, since being in the object tree is
a prerequisite for showing up in the property editor.

Document new texture api

Use the new api to release resources

When we unrealize a GL area with outstanding textures,
call gdk_texture_release_gl on them to avoid later crashes.

Add a way to release GL resources

The inspector may hold on to render nodes and textures
beyond the lifetime of the widget (and thus the GL
resources). To handle this situation, allow the widget
to explicitly release the GL resources, and make
the texture available on the clent-side as a cairo
surface. This lets the recorder still show the content
after the widget is gone.

glarea: Drop the dispose vfunc

This was causing us to leak, in the following scenario:
1) gtk_widget_destroy is called on a GL area
2) dispose is run and clears the context
3) the GL area is unrealized, but the context is already cleared,
   so we leak all the GL buffers

glarea: Always delete textures on unrealize

We need to make the context current for deleting
textures as well, so just do it unconditionally.

glarea: Avoid a crash

We must only delete the texture if it is not NULL.

gdkgears: add a way to remove gears

This is needed to test the GL resource cleanup
code paths.

glarea: Add a pool for GL textures

Handle the situation that a GL texture might remain
in use (e.g. by a slow frame, or by the recorder)
In that case, we can't modify it but must use a
new one. Keep a pool of GL textures for this eventuality.

inspector: Don't leak recordings

g_list_model_get_item is transfer full, so we need
to drop the references we get from it. This was showing
up while testing the GL texture cache in GtkGLArea.

rendernodeimpl: Fix a crash

We may get a NULL renderer here.

applicationwindow: Fix measure implementation

Really calculate the menubar height, not the width.

sizerequest: Fix for_size adjustment

We need to pull the proper size from the size request cache and adjust
it accordingly.

box: Simplify some code

All out varibles in measure implementations must be non-null and -1 is
the default value for the baseline parameters.

box: Remove self assignment

box: Remove useless if statement

We already check right before this one whether child->pack != packing
and if so, we continue to the next iteration. So, no need to check again
whether the inverted condition child->pack == packing is true, because
it is.

label: Update layout width directly form allocation

For the one update_layout_width call in size_allocate, we can just use
the passed-in allocation width instead of a separate (relatively slow)
gtk_widget_get_width call.

widget: Clip redrawn area to widget clip

The clip can already be modified in size_allocate, so we never want a
widget to invalidate more than its actual clip region.

box: Remove some unneeded checks

We only call this function from the measure implementation and we should
never get NULL pointers there.

box: Remove an unnecessary local variable

box: Use GtkWidget's widget list in get_size

listbox: Don't assume that row child == focus widget

With widgets supporting child widgets and custom GtkListBoxRow
subclasses, this might not be true.

widget: Remove reparent special case

Since gtk_widget_reparent doesn't exist anymore, we can also remove this
special case.

container: Remove some unused includes

viewport: Remove some unused includes

gl area: Avoid trivialities

If we have zero pixels, no need to snapshot anything,
and produce GL errors while doing so.

gtk-demo: Add gears to the bowl

Just for fun.

Drop GtkGLArea::has-alpha

We now always behave as if has-alpha is TRUE.

Update all callers.

Convert gtkglarea to use snapshots

Just append a texture node.

Note that this is not 100% done yet. The GL area really
needs to keep a pool of textures, and only reuse them
once the GdkTexture object is gone.

gsk: Handle GL textures

These textures already have a GL texture in them,
no need to create a new one.

gdk: Add a gl texture implementation

This will be used to pass a GL textures from the application
(or rather, GtkGLArea) down to the GSK GL renderer.

Drop removed api from the docs

This should have been part of the previous commit.

gdk: Drop some unused cursor apis

The query function for cursor sizes and capabilities
are not very interesting. At least, they are not used
in GTK+, and all backends but X11 just hardcode
made-up values anyway. So, lets drop them.

inspector: Stop using gdk_display_get_default_cursor_size

If the cursor theme size setting has no value, just hardcode 32.
Also, allow ridiculously large cursors, just for fun.

tooltip: Stop using gdk_display_get_default_cursor_size

It makes more sense to use the actual cursor theme size here.

testgtk: Stop using gdk_display_get_maximal_cursor_size

This function is going away. Just make a reasonable assumption
that cursor sizes up to 128 are supported.

gtk/queryimmodules.c: Make the output deterministic.

Whilst working on the Reproducible Builds effort [0], we noticed that
queryimmodules generates non-reproducible output as it iterates over the
filesystem without sorting.

Patch attached.

 [0] https://reproducible-builds.org/

Signed-off-by: Chris Lamb <lamby@debian.org>
https://bugzilla.gnome.org/show_bug.cgi?id=786528

tree model sort: Fix set_sort_column

We were failing to change the sort order for the
default sort column in some cases. Fix that, and
add a testcase for this issue.

https://bugzilla.gnome.org/show_bug.cgi?id=792459

Add a testcase for the previous fix

tree model sort: Fix initial default sort func

gtk_tree_sortable_has_default_sort_func should return
FALSE initially.

Fix the icontheme test

We rely on log messages here. Since logging is per-display
now, we need to set a display on our custom icontheme object
to get the expected log messages.

mountoperation: Fix a crash

We only create the gesture when the dialog is needed,
so don't free it unconditionally.

Drop ::button-press/release event

We are no longer using these, and there are several gestures
that can be used instead. If you need to catch raw button events,
use the ::event signal.

popover: Stop using ::button-press/release-event

Just use a generic ::event handler.

menushell: Stop using ::button-press/release-event

Merge the handlers into the existing handler for ::event.

menu: Remove stray references to 'button_press'

No code change, but this string was still used in
docs and comments.

places sidebar: Stop using ::button-press/release-event

We'll use a generic ::event signal instead.

gtk-demo: Stop using ::button-release-event

tests: Stop using ::button-press/release-event

The generic ::event signal works fine here.

testsuite: Stop using ::button-press-event

We can use the generic ::event signal instead.

tests: Stop using ::button-press-event

A gesture will do here.

Drop ::motion-notify-event

We no longer emit this signal. You can use various gestures
and event controllers instead. If you need to catch raw
motion events, use the generic ::event signal.

places sidebar: Stop using ::motion-notify-event

We can use the generic ::event signal here.

docs: Stop referring to specific event signals

These are going away.

gtk-demo: Stop using ::motion-notify-event in the hypertext demo

We already have a generic ::event handler, just use it for this.

tests: Stop using ::motion-notify-event

We can use the generic ::event signal here.

treeview: Stop using ::button-press-event

We can just use a gesture here.

recent chooser: Stop using ::button-press-event

We can just use a multipress gesture here.

mount operation: Stop using ::button-press-event

A multipress gesture does just fine here.

testinput: Port to a drag gesture

This replaces both button-press-event and motion-notify-event
handlers.

entry completion: Drop another ::button-press-event handler

This one will be replaced by a generic ::event handler.

docs: Stop referring to legacy event signals

These are going away.

entry completion: Stop using ::button-press-event

Just use the existing ::row-activated signal on the treeviews.

gtk-demo: Stop using ::button-press-event

Use gestures instead.

testgtk: Stop using ::button-press-event

This one example can just as well use a gesture.

places view: Stop using ::button-press-event

Instead, use a multi-press gesture in the row widget,
and emit ::popup-menu from there.

file chooser: Use a multipress gesture

This replaces the use of ::button-press-event. There's two
issues with this commit:
1) We don't have a good way to do the equivalent of
   gdk_event_triggers_context_menu with gestures
2) We have to defer to and idle to avoid ordering
   issues with the treeviews own gestures

Revert "file chooser: Allow activating without double-click"

This reverts commit fb0a13b7f070a14312dafa1e4df6ba03cf33be01.

It is getting in the way of gesture conversion, and didn't
really make anybody happy anyway.

app chooser: Use a gesture

Instead of connecting to ::button-press-event, use
a multipress gesture here.

Remove a leftover icon-size property

This made the 'style classes' demo crash.

Drop ::focus-in/out-event

These signals are no longer used in GTK+.
If you need to catch focus events, use the ::event signal.

treeview: Stop using ::focus-out-event

We can just use the generic ::event signal.

file chooser entry: Stop using ::focus-out-event

Just use the generic ::event signal.

spin button: Stop using ::focus-out-event

Just use the generic ::event signal.

calendar: Stop using ::focus-out-event

Use the generic ::event signal.

color editor: stop using ::focus-out-event

Again, the has-focus property is sufficient.

text cell renderer: Stop using ::focus-out-event

The has-focus property works fine here.

spin cell renderer: Stop using ::focus-out-event

The has-focus property works fine here.

combo cell renderer: Stop using ::focus-out-event

The has-focus property works fine here.

tests: Stop using ::focus-in/out-event

The has-focus property works just as well here.

popover: Stop using ::focus-in/out-event

We can use the is-active property of the window instead.

application: Stop using ::focus-in-event

We are really interested in the active window, so use
that property directly.

window: Stop using ::focus-in/out-event

We can just use the generic ::event.

text view: Stop using ::focus-in/out-event

We can just use the generic ::event here.